Skip to content

enhance: add functions for daemon tools to do mTLS#103

Merged
g-linville merged 1 commit intogptscript-ai:mainfrom
g-linville:mtls
Dec 16, 2024
Merged

enhance: add functions for daemon tools to do mTLS#103
g-linville merged 1 commit intogptscript-ai:mainfrom
g-linville:mtls

Conversation

@g-linville
Copy link
Member

@g-linville g-linville commented Dec 4, 2024 •
edited
Loading

(The test failures on this PR are nothing new.)

This implementation is compatible with Express, which is what we use in our Node-based daemon tools. The app that Express creates is compatible with the listener argument that the createServer function takes.

Example usage:

import { createServer, startServer } from '@gptscript-ai/gptscript'

const server = createServer((req, res) => {
    res.writeHead(200)
    res.end('Howdy!')
})

try {
    startServer(server)
} catch (e) {
    console.log(e)
}

@g-linville g-linville marked this pull request as draft December 4, 2024 21:33
@g-linville g-linville marked this pull request as ready for review December 6, 2024 03:32
@g-linville g-linville merged commit 74e083d into gptscript-ai:main Dec 16, 2024
1 of 2 checks passed
@g-linville g-linville deleted the mtls branch December 16, 2024 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@drpebcak drpebcak drpebcak approved these changes

@thedadams thedadams Awaiting requested review from thedadams

@njhale njhale Awaiting requested review from njhale

@StrongMonkey StrongMonkey Awaiting requested review from StrongMonkey

+1 more reviewer

@iwilltry42 iwilltry42 iwilltry42 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@g-linville @drpebcak @iwilltry42